summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorlat9nq <22451773+lat9nq@users.noreply.github.com>2023-06-09 22:53:26 +0200
committerlat9nq <22451773+lat9nq@users.noreply.github.com>2023-07-21 16:56:07 +0200
commit4ff8255e4a985e69046e453a9bd38adf80346548 (patch)
tree9d665f43b79639047e11dfda63f4036885619204
parentsettings, uisettings: Initialize linkage counter (diff)
downloadyuzu-4ff8255e4a985e69046e453a9bd38adf80346548.tar
yuzu-4ff8255e4a985e69046e453a9bd38adf80346548.tar.gz
yuzu-4ff8255e4a985e69046e453a9bd38adf80346548.tar.bz2
yuzu-4ff8255e4a985e69046e453a9bd38adf80346548.tar.lz
yuzu-4ff8255e4a985e69046e453a9bd38adf80346548.tar.xz
yuzu-4ff8255e4a985e69046e453a9bd38adf80346548.tar.zst
yuzu-4ff8255e4a985e69046e453a9bd38adf80346548.zip
-rw-r--r--src/yuzu/configuration/configure_graphics.cpp9
-rw-r--r--src/yuzu/configuration/configure_graphics.h4
-rw-r--r--src/yuzu/configuration/configure_system.cpp23
-rw-r--r--src/yuzu/configuration/shared_translation.cpp8
-rw-r--r--src/yuzu/configuration/shared_widget.cpp395
-rw-r--r--src/yuzu/configuration/shared_widget.h35
6 files changed, 254 insertions, 220 deletions
diff --git a/src/yuzu/configuration/configure_graphics.cpp b/src/yuzu/configuration/configure_graphics.cpp
index 977aed42d..2354323b8 100644
--- a/src/yuzu/configuration/configure_graphics.cpp
+++ b/src/yuzu/configuration/configure_graphics.cpp
@@ -165,6 +165,11 @@ void ConfigureGraphics::PopulateVSyncModeSelection() {
: vsync_mode_combobox_enum_map[current_index];
int index{};
const int device{vulkan_device_combobox->currentIndex()}; //< current selected Vulkan device
+ if (device == -1) {
+ // Invalid device
+ return;
+ }
+
const auto& present_modes = //< relevant vector of present modes for the selected device or API
backend == Settings::RendererBackend::Vulkan ? device_present_modes[device]
: default_present_modes;
@@ -236,11 +241,11 @@ void ConfigureGraphics::Setup() {
return new ConfigurationShared::Widget(
setting, translations, this, runtime_lock, apply_funcs,
ConfigurationShared::RequestType::ReverseSlider, true, 0.5f);
- } else if (setting->Id() == Settings::values.use_speed_limit.Id()) {
+ } else if (setting->Id() == Settings::values.speed_limit.Id()) {
return new ConfigurationShared::Widget(
setting, translations, this, runtime_lock, apply_funcs,
ConfigurationShared::RequestType::SpinBox, true, 1.0f,
- &Settings::values.speed_limit, "%");
+ &Settings::values.use_speed_limit, "%");
} else {
return new ConfigurationShared::Widget(setting, translations, this, runtime_lock,
apply_funcs);
diff --git a/src/yuzu/configuration/configure_graphics.h b/src/yuzu/configuration/configure_graphics.h
index 61eb2f2fc..f36495ed3 100644
--- a/src/yuzu/configuration/configure_graphics.h
+++ b/src/yuzu/configuration/configure_graphics.h
@@ -77,8 +77,8 @@ private:
std::vector<QString> vulkan_devices;
std::vector<std::vector<VkPresentModeKHR>> device_present_modes;
std::vector<VkPresentModeKHR>
- vsync_mode_combobox_enum_map; //< Keeps track of which present mode corresponds to which
- // selection in the combobox
+ vsync_mode_combobox_enum_map{}; //< Keeps track of which present mode corresponds to which
+ // selection in the combobox
u32 vulkan_device{};
Settings::ShaderBackend shader_backend{};
const std::function<void()>& expose_compute_option;
diff --git a/src/yuzu/configuration/configure_system.cpp b/src/yuzu/configuration/configure_system.cpp
index dedbad57f..128860800 100644
--- a/src/yuzu/configuration/configure_system.cpp
+++ b/src/yuzu/configuration/configure_system.cpp
@@ -117,17 +117,18 @@ void ConfigureSystem::Setup() {
push(Settings::values.linkage.by_category[Settings::Category::System]);
for (auto setting : settings) {
+ [[maybe_unused]] std::string label = setting->GetLabel();
ConfigurationShared::Widget* widget = [=]() {
- if (setting->Id() == Settings::values.custom_rtc_enabled.Id()) {
+ if (setting->Id() == Settings::values.custom_rtc.Id()) {
return new ConfigurationShared::Widget(
setting, translations, this, runtime_lock, apply_funcs,
ConfigurationShared::RequestType::DateTimeEdit, true, 1.0f,
- &Settings::values.custom_rtc);
- } else if (setting->Id() == Settings::values.rng_seed_enabled.Id()) {
- return new ConfigurationShared::Widget(setting, translations, this, runtime_lock,
- apply_funcs,
- ConfigurationShared::RequestType::HexEdit,
- true, 1.0f, &Settings::values.rng_seed);
+ &Settings::values.custom_rtc_enabled);
+ } else if (setting->Id() == Settings::values.rng_seed.Id()) {
+ return new ConfigurationShared::Widget(
+ setting, translations, this, runtime_lock, apply_funcs,
+ ConfigurationShared::RequestType::HexEdit, true, 1.0f,
+ &Settings::values.rng_seed_enabled);
} else {
return new ConfigurationShared::Widget(setting, translations, this, runtime_lock,
@@ -140,14 +141,12 @@ void ConfigureSystem::Setup() {
continue;
}
- if (setting->Id() == Settings::values.rng_seed_enabled.Id()) {
+ if (setting->Id() == Settings::values.rng_seed.Id()) {
rng_seed_checkbox = widget->checkbox;
rng_seed_edit = widget->line_edit;
- if (!Settings::values.rng_seed_enabled.GetValue()) {
- rng_seed_edit->setEnabled(false);
- }
- } else if (setting->Id() == Settings::values.custom_rtc_enabled.Id()) {
+ rng_seed_edit->setEnabled(Settings::values.rng_seed_enabled.GetValue());
+ } else if (setting->Id() == Settings::values.custom_rtc.Id()) {
custom_rtc_checkbox = widget->checkbox;
custom_rtc_edit = widget->date_time_edit;
diff --git a/src/yuzu/configuration/shared_translation.cpp b/src/yuzu/configuration/shared_translation.cpp
index dc9f15cdd..6038e8c25 100644
--- a/src/yuzu/configuration/shared_translation.cpp
+++ b/src/yuzu/configuration/shared_translation.cpp
@@ -108,11 +108,11 @@ std::unique_ptr<TranslationMap> InitializeTranslations(QWidget* parent) {
INSERT(Settings, speed_limit, "Limit Speed Percent", "");
// System
- INSERT(Settings, rng_seed_enabled, "RNG Seed", "");
- INSERT(Settings, rng_seed, "", "");
+ INSERT(Settings, rng_seed, "RNG Seed", "");
+ INSERT(Settings, rng_seed_enabled, "", "");
INSERT(Settings, device_name, "Device Name", "");
- INSERT(Settings, custom_rtc_enabled, "Custom RTC", "");
- INSERT(Settings, custom_rtc, "", "");
+ INSERT(Settings, custom_rtc, "Custom RTC", "");
+ INSERT(Settings, custom_rtc_enabled, "", "");
INSERT(Settings, language_index, "Language:", "");
INSERT(Settings, region_index, "Region:", "");
INSERT(Settings, time_zone_index, "Time Zone:", "");
diff --git a/src/yuzu/configuration/shared_widget.cpp b/src/yuzu/configuration/shared_widget.cpp
index 0d553c67f..3ef2c25c6 100644
--- a/src/yuzu/configuration/shared_widget.cpp
+++ b/src/yuzu/configuration/shared_widget.cpp
@@ -12,7 +12,9 @@
#include <QWidget>
#include <qabstractbutton.h>
#include <qabstractspinbox.h>
+#include <qboxlayout.h>
#include <qnamespace.h>
+#include <qpushbutton.h>
#include <qvalidator.h>
#include "common/common_types.h"
#include "common/settings.h"
@@ -22,10 +24,6 @@
namespace ConfigurationShared {
-static bool IsInt(const std::type_index& type) {
- return type == typeid(u32) || type == typeid(s32) || type == typeid(u16) || type == typeid(s16);
-}
-
QPushButton* Widget::CreateRestoreGlobalButton(Settings::BasicSetting& setting, QWidget* parent) {
QStyle* style = parent->style();
QIcon* icon = new QIcon(style->standardIcon(QStyle::SP_DialogResetButton));
@@ -42,50 +40,65 @@ QPushButton* Widget::CreateRestoreGlobalButton(Settings::BasicSetting& setting,
return restore_button;
}
-void Widget::CreateCheckBox(const QString& label, std::function<void()>& load_func) {
+QLabel* Widget::CreateLabel(const QString& text) {
+ QLabel* qt_label = new QLabel(text, this->parent);
+ qt_label->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Preferred);
+ return qt_label;
+}
+
+QHBoxLayout* Widget::CreateCheckBox(Settings::BasicSetting* bool_setting, const QString& label,
+ std::function<void()>& load_func, bool managed) {
created = true;
QHBoxLayout* layout = new QHBoxLayout(this);
checkbox = new QCheckBox(label, this);
- checkbox->setObjectName(QString::fromStdString(setting.GetLabel()));
- checkbox->setCheckState(setting.ToString() == "true" ? Qt::CheckState::Checked
- : Qt::CheckState::Unchecked);
+ checkbox->setCheckState(bool_setting->ToString() == "true" ? Qt::CheckState::Checked
+ : Qt::CheckState::Unchecked);
+ checkbox->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Preferred);
layout->addWidget(checkbox);
+
+ layout->setContentsMargins(0, 0, 0, 0);
+
+ if (!managed) {
+ return layout;
+ }
+
if (Settings::IsConfiguringGlobal()) {
load_func = [=]() {
- setting.LoadString(checkbox->checkState() == Qt::Checked ? "true" : "false");
+ bool_setting->LoadString(checkbox->checkState() == Qt::Checked ? "true" : "false");
};
} else {
- restore_button = CreateRestoreGlobalButton(setting, this);
+ restore_button = CreateRestoreGlobalButton(*bool_setting, this);
layout->addWidget(restore_button);
- QObject::connect(checkbox, &QCheckBox::stateChanged, [&](int) {
+ QObject::connect(checkbox, &QCheckBox::stateChanged, [=](int) {
restore_button->setVisible(true);
restore_button->setEnabled(true);
});
- QObject::connect(restore_button, &QAbstractButton::clicked, [&](bool) {
- checkbox->setCheckState(setting.ToStringGlobal() == "true" ? Qt::Checked
- : Qt::Unchecked);
+ QObject::connect(restore_button, &QAbstractButton::clicked, [=](bool) {
+ checkbox->setCheckState(bool_setting->ToStringGlobal() == "true" ? Qt::Checked
+ : Qt::Unchecked);
restore_button->setEnabled(false);
restore_button->setVisible(false);
});
load_func = [=]() {
bool using_global = !restore_button->isEnabled();
- setting.SetGlobal(using_global);
+ bool_setting->SetGlobal(using_global);
if (!using_global) {
- setting.LoadString(checkbox->checkState() == Qt::Checked ? "true" : "false");
+ bool_setting->LoadString(checkbox->checkState() == Qt::Checked ? "true" : "false");
}
};
}
- layout->setContentsMargins(0, 0, 0, 0);
+ return layout;
}
-void Widget::CreateCombobox(const QString& label, bool managed, std::function<void()>& load_func) {
+void Widget::CreateCombobox(const QString& label, std::function<void()>& load_func, bool managed,
+ Settings::BasicSetting* const other_setting) {
created = true;
const auto type = setting.TypeId();
@@ -108,9 +121,13 @@ void Widget::CreateCombobox(const QString& label, bool managed, std::function<vo
combobox->setCurrentIndex(std::stoi(setting.ToString()));
- if (Settings::IsConfiguringGlobal() && managed) {
+ if (!managed) {
+ return;
+ }
+
+ if (Settings::IsConfiguringGlobal()) {
load_func = [=]() { setting.LoadString(std::to_string(combobox->currentIndex())); };
- } else if (managed) {
+ } else {
restore_button = CreateRestoreGlobalButton(setting, this);
layout->addWidget(restore_button);
@@ -136,30 +153,50 @@ void Widget::CreateCombobox(const QString& label, bool managed, std::function<vo
}
}
-void Widget::CreateLineEdit(const QString& label, bool managed, std::function<void()>& load_func) {
+void Widget::CreateLineEdit(const QString& label, std::function<void()>& load_func, bool managed,
+ Settings::BasicSetting* other_setting) {
+ const bool has_checkbox = other_setting != nullptr;
+ if (has_checkbox && other_setting->TypeId() != typeid(bool)) {
+ LOG_WARNING(Frontend, "Extra setting requested but setting is not boolean");
+ return;
+ }
+
created = true;
- QHBoxLayout* layout = new QHBoxLayout(this);
- line_edit = new QLineEdit(this);
+ QHBoxLayout* layout{nullptr};
+ std::function<void()> checkbox_load_func = []() {};
+
+ if (has_checkbox) {
+ layout = CreateCheckBox(other_setting, label, checkbox_load_func, managed);
+ } else {
+ layout = new QHBoxLayout(this);
+ layout->setContentsMargins(0, 0, 0, 0);
+ QLabel* q_label = CreateLabel(label);
+ layout->addWidget(q_label);
+ }
const QString text = QString::fromStdString(setting.ToString());
+ line_edit = new QLineEdit(this);
line_edit->setText(text);
- QLabel* q_label = new QLabel(label, this);
- // setSizePolicy lets widget expand and take an equal part of the space as the line edit
- q_label->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Preferred);
- layout->addWidget(q_label);
-
layout->addWidget(line_edit);
- if (Settings::IsConfiguringGlobal() && !managed) {
+ if (!managed) {
+ return;
+ }
+
+ if (Settings::IsConfiguringGlobal()) {
load_func = [=]() {
+ checkbox_load_func();
+
std::string load_text = line_edit->text().toStdString();
setting.LoadString(load_text);
};
- } else if (!managed) {
- restore_button = CreateRestoreGlobalButton(setting, this);
- layout->addWidget(restore_button);
+ } else {
+ if (!has_checkbox) {
+ restore_button = CreateRestoreGlobalButton(setting, this);
+ layout->addWidget(restore_button);
+ }
QObject::connect(restore_button, &QAbstractButton::clicked, [&](bool) {
restore_button->setEnabled(false);
@@ -174,6 +211,8 @@ void Widget::CreateLineEdit(const QString& label, bool managed, std::function<vo
});
load_func = [=]() {
+ checkbox_load_func();
+
bool using_global = !restore_button->isEnabled();
setting.SetGlobal(using_global);
if (!using_global) {
@@ -181,24 +220,23 @@ void Widget::CreateLineEdit(const QString& label, bool managed, std::function<vo
}
};
}
-
- layout->setContentsMargins(0, 0, 0, 0);
}
-void Widget::CreateSlider(const QString& name, bool reversed, float multiplier,
- std::function<void()>& load_func) {
+void Widget::CreateSlider(const QString& label, bool reversed, float multiplier,
+ std::function<void()>& load_func, bool managed,
+ Settings::BasicSetting* const other_setting) {
created = true;
QHBoxLayout* layout = new QHBoxLayout(this);
slider = new QSlider(Qt::Horizontal, this);
- QLabel* label = new QLabel(name, this);
+ QLabel* qt_label = new QLabel(label, this);
QLabel* feedback = new QLabel(this);
- layout->addWidget(label);
+ layout->addWidget(qt_label);
layout->addWidget(slider);
layout->addWidget(feedback);
- label->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Preferred);
+ qt_label->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Preferred);
layout->setContentsMargins(0, 0, 0, 0);
@@ -214,8 +252,10 @@ void Widget::CreateSlider(const QString& name, bool reversed, float multiplier,
slider->setMinimum(std::stoi(setting.MinVal()));
slider->setMaximum(max_val);
- if (reversed) {
- slider->setInvertedAppearance(true);
+ slider->setInvertedAppearance(reversed);
+
+ if (!managed) {
+ return;
}
if (Settings::IsConfiguringGlobal()) {
@@ -246,51 +286,54 @@ void Widget::CreateSlider(const QString& name, bool reversed, float multiplier,
}
}
-void Widget::CreateCheckBoxWithHexEdit(const QString& label, Settings::BasicSetting* other_setting,
- std::function<void()>& load_func) {
- if (other_setting == nullptr) {
- LOG_WARNING(Frontend, "Extra setting is null or not an integer");
+void Widget::CreateSpinBox(const QString& label, std::function<void()>& load_func, bool managed,
+ const std::string& suffix, Settings::BasicSetting* other_setting) {
+ const bool has_checkbox = other_setting != nullptr;
+ if (has_checkbox && other_setting->TypeId() != typeid(bool)) {
+ LOG_WARNING(Frontend, "Extra setting requested but setting is not boolean");
return;
}
created = true;
- std::function<void()> checkbox_load_func;
- CreateCheckBox(label, checkbox_load_func);
-
- auto to_hex = [=](const std::string& input) {
- return QString::fromStdString(fmt::format("{:08x}", std::stoi(input)));
- };
-
- QHBoxLayout* layout = reinterpret_cast<QHBoxLayout*>(this->layout());
- const QString default_val = to_hex(other_setting->ToString());
+ QHBoxLayout* layout{nullptr};
+ std::function<void()> checkbox_load_func = []() {};
+ QLabel* q_label{nullptr};
- line_edit = new QLineEdit(this);
- line_edit->setText(default_val);
-
- checkbox->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Preferred);
+ if (has_checkbox) {
+ layout = CreateCheckBox(other_setting, label, checkbox_load_func, managed);
+ } else {
+ layout = new QHBoxLayout(this);
+ layout->setContentsMargins(0, 0, 0, 0);
+ q_label = CreateLabel(label);
+ layout->addWidget(q_label);
+ }
- layout->insertWidget(1, line_edit);
+ const int min_val = std::stoi(setting.MinVal());
+ const int max_val = std::stoi(setting.MaxVal());
+ const int default_val = std::stoi(setting.ToString());
- line_edit->setMaxLength(8);
- QRegExpValidator* regex =
- new QRegExpValidator{QRegExp{QStringLiteral("^[0-9a-fA-F]{0,8}$")}, line_edit};
- line_edit->setValidator(regex);
+ spinbox = new QSpinBox(this);
+ spinbox->setRange(min_val, max_val);
+ spinbox->setValue(default_val);
+ spinbox->setSuffix(QString::fromStdString(suffix));
+ spinbox->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Preferred);
- auto hex_to_dec = [=]() -> std::string {
- return std::to_string(std::stoul(line_edit->text().toStdString(), nullptr, 16));
- };
+ layout->insertWidget(1, spinbox);
if (Settings::IsConfiguringGlobal()) {
load_func = [=]() {
checkbox_load_func();
- other_setting->LoadString(hex_to_dec());
+ setting.LoadString(std::to_string(spinbox->value()));
};
} else {
- QObject::connect(restore_button, &QAbstractButton::clicked, [=](bool) {
- line_edit->setText(to_hex(other_setting->ToStringGlobal()));
- });
+ if (!has_checkbox) {
+ restore_button = CreateRestoreGlobalButton(setting, this);
+ }
+
+ QObject::connect(restore_button, &QAbstractButton::clicked,
+ [this](bool) { spinbox->setValue(std::stoi(setting.ToStringGlobal())); });
- QObject::connect(line_edit, &QLineEdit::textEdited, [=](const QString&) {
+ QObject::connect(spinbox, QOverload<int>::of(&QSpinBox::valueChanged), [this](int) {
restore_button->setEnabled(true);
restore_button->setVisible(true);
});
@@ -299,148 +342,122 @@ void Widget::CreateCheckBoxWithHexEdit(const QString& label, Settings::BasicSett
checkbox_load_func();
const bool using_global = !restore_button->isEnabled();
- other_setting->SetGlobal(using_global);
+ setting.SetGlobal(using_global);
if (!using_global) {
- other_setting->LoadString(hex_to_dec());
+ setting.LoadString(std::to_string(spinbox->value()));
}
};
}
}
-void Widget::CreateCheckBoxWithLineEdit(const QString& label, Settings::BasicSetting* other_setting,
- std::function<void()>& load_func) {
- if (other_setting == nullptr) {
- LOG_WARNING(Frontend, "Extra setting is null or not an integer");
+void Widget::CreateHexEdit(const QString& label, std::function<void()>& load_func, bool managed,
+ Settings::BasicSetting* const other_setting) {
+ CreateLineEdit(label, load_func, false, other_setting);
+ if (!created || !managed) {
return;
}
- created = true;
- std::function<void()> checkbox_load_func;
- CreateCheckBox(label, checkbox_load_func);
+ QLayout* layout = this->layout();
+
+ auto to_hex = [=](const std::string& input) {
+ return QString::fromStdString(fmt::format("{:08x}", std::stoi(input)));
+ };
+
+ QRegExpValidator* regex =
+ new QRegExpValidator{QRegExp{QStringLiteral("^[0-9a-fA-F]{0,8}$")}, line_edit};
- QHBoxLayout* layout = reinterpret_cast<QHBoxLayout*>(this->layout());
- const QString default_val = QString::fromStdString(other_setting->ToString());
+ const QString default_val = to_hex(setting.ToString());
- line_edit = new QLineEdit(this);
line_edit->setText(default_val);
+ line_edit->setMaxLength(8);
+ line_edit->setValidator(regex);
- checkbox->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Preferred);
-
- layout->insertWidget(1, line_edit);
+ auto hex_to_dec = [=]() -> std::string {
+ return std::to_string(std::stoul(line_edit->text().toStdString(), nullptr, 16));
+ };
if (Settings::IsConfiguringGlobal()) {
load_func = [=]() {
- checkbox_load_func();
- other_setting->LoadString(line_edit->text().toStdString());
+ other_setting->LoadString(checkbox->checkState() == Qt::Checked ? "true" : "false");
+ setting.LoadString(hex_to_dec());
};
} else {
+ restore_button = CreateRestoreGlobalButton(setting, this);
+ layout->addWidget(restore_button);
+
QObject::connect(restore_button, &QAbstractButton::clicked, [=](bool) {
- line_edit->setText(QString::fromStdString(other_setting->ToStringGlobal()));
+ line_edit->setText(to_hex(setting.ToStringGlobal()));
+ checkbox->setCheckState(other_setting->ToStringGlobal() == "true" ? Qt::Checked
+ : Qt::Unchecked);
+
+ restore_button->setEnabled(false);
+ restore_button->setVisible(false);
});
- QObject::connect(line_edit, &QLineEdit::textEdited, [=](const QString&) {
+ QObject::connect(line_edit, &QLineEdit::textEdited, [&]() {
restore_button->setEnabled(true);
restore_button->setVisible(true);
});
- load_func = [=]() {
- checkbox_load_func();
-
- const bool using_global = !restore_button->isEnabled();
- other_setting->SetGlobal(using_global);
- if (!using_global) {
- other_setting->LoadString(line_edit->text().toStdString());
- }
- };
- }
-}
-
-void Widget::CreateCheckBoxWithSpinBox(const QString& label, Settings::BasicSetting* other_setting,
- std::function<void()>& load_func,
- const std::string& suffix) {
- if (other_setting == nullptr && IsInt(other_setting->TypeId())) {
- LOG_WARNING(Frontend, "Extra setting is null or not an integer");
- return;
- }
- created = true;
-
- std::function<void()> checkbox_load_func;
- CreateCheckBox(label, checkbox_load_func);
- checkbox->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Preferred);
-
- QHBoxLayout* layout = reinterpret_cast<QHBoxLayout*>(this->layout());
-
- spinbox = new QSpinBox(this);
- const int min_val = std::stoi(other_setting->MinVal());
- const int max_val = std::stoi(other_setting->MaxVal());
- const int default_val = std::stoi(other_setting->ToString());
- spinbox->setRange(min_val, max_val);
- spinbox->setValue(default_val);
- spinbox->setSuffix(QString::fromStdString(suffix));
- spinbox->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Preferred);
-
- layout->insertWidget(1, spinbox);
-
- if (Settings::IsConfiguringGlobal()) {
- load_func = [=]() {
- checkbox_load_func();
- other_setting->LoadString(std::to_string(spinbox->value()));
- };
- } else {
- QObject::connect(restore_button, &QAbstractButton::clicked, [this, other_setting](bool) {
- spinbox->setValue(std::stoi(other_setting->ToStringGlobal()));
- });
-
- QObject::connect(spinbox, QOverload<int>::of(&QSpinBox::valueChanged), [this](int) {
+ QObject::connect(checkbox, &QAbstractButton::clicked, [&]() {
restore_button->setEnabled(true);
restore_button->setVisible(true);
});
load_func = [=]() {
- checkbox_load_func();
-
const bool using_global = !restore_button->isEnabled();
other_setting->SetGlobal(using_global);
+ setting.SetGlobal(using_global);
+
if (!using_global) {
- other_setting->LoadString(std::to_string(spinbox->value()));
+ other_setting->LoadString(checkbox->checkState() == Qt::Checked ? "true" : "false");
+ setting.LoadString(hex_to_dec());
}
};
}
}
-// Currently tailored to custom_rtc
-void Widget::CreateCheckBoxWithDateTimeEdit(const QString& label,
- Settings::BasicSetting* other_setting,
- std::function<void()>& load_func) {
- if (other_setting == nullptr) {
- LOG_WARNING(Frontend, "Extra setting is null or not an integer");
+void Widget::CreateDateTimeEdit(const QString& label, std::function<void()>& load_func,
+ bool managed, bool restrict,
+ Settings::BasicSetting* const other_setting) {
+ const bool has_checkbox = other_setting != nullptr;
+ if ((restrict && !has_checkbox) || (has_checkbox && other_setting->TypeId() != typeid(bool))) {
+ LOG_WARNING(Frontend, "Extra setting or restrict requested but is not boolean");
return;
}
created = true;
- std::function<void()> checkbox_load_func;
- CreateCheckBox(label, checkbox_load_func);
+ QHBoxLayout* layout{nullptr};
+ std::function<void()> checkbox_load_func = []() {};
+
+ if (has_checkbox) {
+ layout = CreateCheckBox(other_setting, label, checkbox_load_func, managed);
+ } else {
+ layout = new QHBoxLayout(this);
+ QLabel* q_label = CreateLabel(label);
+ layout->addWidget(q_label);
+ }
- QHBoxLayout* layout = reinterpret_cast<QHBoxLayout*>(this->layout());
- const bool disabled = setting.ToString() != "true";
+ const bool disabled = other_setting->ToString() != "true";
const long long current_time = QDateTime::currentSecsSinceEpoch();
- const s64 the_time = disabled ? current_time : std::stoll(other_setting->ToString());
+ const s64 the_time = disabled ? current_time : std::stoll(setting.ToString());
const auto default_val = QDateTime::fromSecsSinceEpoch(the_time);
date_time_edit = new QDateTimeEdit(this);
date_time_edit->setDateTime(default_val);
-
date_time_edit->setMinimumDateTime(QDateTime::fromSecsSinceEpoch(0));
-
date_time_edit->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Preferred);
- checkbox->setSizePolicy(QSizePolicy::Expanding, QSizePolicy::Preferred);
layout->insertWidget(1, date_time_edit);
+ if (!managed) {
+ return;
+ }
+
if (Settings::IsConfiguringGlobal()) {
load_func = [=]() {
checkbox_load_func();
- if (checkbox->checkState() == Qt::Unchecked) {
+ if (restrict && checkbox->checkState() == Qt::Unchecked) {
return;
}
@@ -448,9 +465,14 @@ void Widget::CreateCheckBoxWithDateTimeEdit(const QString& label,
std::to_string(date_time_edit->dateTime().toSecsSinceEpoch()));
};
} else {
+ if (!has_checkbox) {
+ restore_button = CreateRestoreGlobalButton(setting, this);
+ layout->addWidget(restore_button);
+ }
+
auto get_clear_val = [=]() {
return QDateTime::fromSecsSinceEpoch([=]() {
- if (checkbox->checkState() == Qt::Checked) {
+ if (restrict && checkbox->checkState() == Qt::Checked) {
return std::stoll(other_setting->ToStringGlobal());
}
return current_time;
@@ -469,7 +491,7 @@ void Widget::CreateCheckBoxWithDateTimeEdit(const QString& label,
load_func = [=]() {
checkbox_load_func();
- if (checkbox->checkState() == Qt::Unchecked) {
+ if (restrict && checkbox->checkState() == Qt::Unchecked) {
return;
}
@@ -490,11 +512,17 @@ bool Widget::Valid() {
Widget::~Widget() = default;
Widget::Widget(Settings::BasicSetting* setting_, const TranslationMap& translations_,
+ QWidget* parent_, std::forward_list<std::function<void(bool)>>& apply_funcs_)
+ : QWidget(parent_), parent{parent_}, translations{translations_}, setting{*setting_},
+ apply_funcs{apply_funcs_} {}
+
+Widget::Widget(Settings::BasicSetting* setting_, const TranslationMap& translations_,
QWidget* parent_, bool runtime_lock,
- std::forward_list<std::function<void(bool)>>& apply_funcs, RequestType request,
+ std::forward_list<std::function<void(bool)>>& apply_funcs_, RequestType request,
bool managed, float multiplier, Settings::BasicSetting* other_setting,
const std::string& string)
- : QWidget(parent_), parent{parent_}, translations{translations_}, setting{*setting_} {
+ : QWidget(parent_), parent{parent_}, translations{translations_}, setting{*setting_},
+ apply_funcs{apply_funcs_} {
if (!Settings::IsConfiguringGlobal() && !setting.Switchable()) {
LOG_DEBUG(Frontend, "\"{}\" is not switchable, skipping...", setting.GetLabel());
return;
@@ -523,51 +551,44 @@ Widget::Widget(Settings::BasicSetting* setting_, const TranslationMap& translati
if (type == typeid(bool)) {
switch (request) {
case RequestType::Default:
- CreateCheckBox(label, load_func);
+ CreateCheckBox(&setting, label, load_func, managed);
break;
- case RequestType::SpinBox:
- CreateCheckBoxWithSpinBox(label, other_setting, load_func, string);
- break;
- case RequestType::HexEdit:
- CreateCheckBoxWithHexEdit(label, other_setting, load_func);
- break;
- case RequestType::LineEdit:
- CreateCheckBoxWithLineEdit(label, other_setting, load_func);
- break;
- case RequestType::DateTimeEdit:
- CreateCheckBoxWithDateTimeEdit(label, other_setting, load_func);
- break;
- case RequestType::ComboBox:
- case RequestType::Slider:
- case RequestType::ReverseSlider:
- case RequestType::MaxEnum:
- LOG_DEBUG(Frontend, "Requested widget is unimplemented.");
+ default:
+ LOG_WARNING(Frontend, "Requested widget is unimplemented.");
break;
}
} else if (setting.IsEnum()) {
- CreateCombobox(label, managed, load_func);
- } else if (type == typeid(u32) || type == typeid(int)) {
+ CreateCombobox(label, load_func, managed);
+ } else if (type == typeid(u32) || type == typeid(int) || type == typeid(u16) ||
+ type == typeid(s64)) {
switch (request) {
case RequestType::Slider:
case RequestType::ReverseSlider:
- CreateSlider(label, request == RequestType::ReverseSlider, multiplier, load_func);
+ CreateSlider(label, request == RequestType::ReverseSlider, multiplier, load_func,
+ managed);
break;
case RequestType::LineEdit:
case RequestType::Default:
- CreateLineEdit(label, managed, load_func);
+ CreateLineEdit(label, load_func, managed);
break;
case RequestType::ComboBox:
- CreateCombobox(label, managed, load_func);
+ CreateCombobox(label, load_func, managed);
break;
case RequestType::DateTimeEdit:
+ CreateDateTimeEdit(label, load_func, managed, true, other_setting);
+ break;
case RequestType::SpinBox:
+ CreateSpinBox(label, load_func, managed, string, other_setting);
+ break;
case RequestType::HexEdit:
- case RequestType::MaxEnum:
- LOG_DEBUG(Frontend, "Requested widget is unimplemented.");
+ CreateHexEdit(label, load_func, managed, other_setting);
+ break;
+ default:
+ LOG_WARNING(Frontend, "Requested widget is unimplemented.");
break;
}
} else if (type == typeid(std::string)) {
- CreateLineEdit(label, managed, load_func);
+ CreateLineEdit(label, load_func, managed);
}
if (!created) {
diff --git a/src/yuzu/configuration/shared_widget.h b/src/yuzu/configuration/shared_widget.h
index 9923aa2ea..c4e686574 100644
--- a/src/yuzu/configuration/shared_widget.h
+++ b/src/yuzu/configuration/shared_widget.h
@@ -9,6 +9,8 @@ class QComboBox;
class QLineEdit;
class QSlider;
class QCheckBox;
+class QLabel;
+class QHBoxLayout;
class QDateTimeEdit;
namespace Settings {
@@ -34,9 +36,11 @@ class Widget : public QWidget {
public:
Widget(Settings::BasicSetting* setting, const TranslationMap& translations, QWidget* parent,
- bool runtime_lock, std::forward_list<std::function<void(bool)>>& apply_funcs,
+ bool runtime_lock, std::forward_list<std::function<void(bool)>>& apply_funcs_,
RequestType request = RequestType::Default, bool managed = true, float multiplier = 1.0f,
Settings::BasicSetting* other_setting = nullptr, const std::string& format = "");
+ Widget(Settings::BasicSetting* setting_, const TranslationMap& translations_, QWidget* parent_,
+ std::forward_list<std::function<void(bool)>>& apply_funcs_);
virtual ~Widget();
bool Valid();
@@ -53,23 +57,28 @@ public:
QDateTimeEdit* date_time_edit{};
private:
- void CreateCheckBox(const QString& label, std::function<void()>& load_func);
- void CreateCheckBoxWithLineEdit(const QString& label, Settings::BasicSetting* other_setting,
- std::function<void()>& load_func);
- void CreateCheckBoxWithHexEdit(const QString& label, Settings::BasicSetting* other_setting,
- std::function<void()>& load_func);
- void CreateCheckBoxWithSpinBox(const QString& label, Settings::BasicSetting* other_setting,
- std::function<void()>& load_func, const std::string& suffix);
- void CreateCheckBoxWithDateTimeEdit(const QString& label, Settings::BasicSetting* other_setting,
- std::function<void()>& load_func);
- void CreateCombobox(const QString& label, bool managed, std::function<void()>& load_func);
- void CreateLineEdit(const QString& label, bool managed, std::function<void()>& load_func);
+ QLabel* CreateLabel(const QString& text);
+ QHBoxLayout* CreateCheckBox(Settings::BasicSetting* bool_setting, const QString& label,
+ std::function<void()>& load_func, bool managed);
+
+ void CreateCombobox(const QString& label, std::function<void()>& load_func, bool managed,
+ Settings::BasicSetting* const other_setting = nullptr);
+ void CreateLineEdit(const QString& label, std::function<void()>& load_func, bool managed,
+ Settings::BasicSetting* const other_setting = nullptr);
+ void CreateHexEdit(const QString& label, std::function<void()>& load_func, bool managed,
+ Settings::BasicSetting* const other_setting = nullptr);
void CreateSlider(const QString& label, bool reversed, float multiplier,
- std::function<void()>& load_func);
+ std::function<void()>& load_func, bool managed,
+ Settings::BasicSetting* const other_setting = nullptr);
+ void CreateDateTimeEdit(const QString& label, std::function<void()>& load_func, bool managed,
+ bool restrict, Settings::BasicSetting* const other_setting = nullptr);
+ void CreateSpinBox(const QString& label, std::function<void()>& load_func, bool managed,
+ const std::string& suffix, Settings::BasicSetting* other_setting = nullptr);
QWidget* parent;
const TranslationMap& translations;
Settings::BasicSetting& setting;
+ std::forward_list<std::function<void(bool)>>& apply_funcs;
bool created{false};
};